EVM instruction interpreter: Add loadimmutable and setimmutable mocks #16387
+20
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…utable Theoretically, loadimmutable inserts a placeholder into the contract to be deployed which is then replaced by setimmutable once the runtime code is loaded into memory. The interpreter is mainly used for differential fuzzing so we should satisfy that correct code never fails and we can still detect bugs otherwise, most of the time. Therefore loadimmutable is decoupled from setimmutable and just returns a random value (hash of the identifier), setimmutable is treated as a no-op.
clonker
force-pushed
the
evm_instruction_interpreter_kiss_immutables
branch
from
7f71643 to
f8cae91
Compare
cameel
reviewed
Jan 14, 2026
Collaborator
There was a problem hiding this comment.
I think it would be good to have at least some minimal smoke tests for the interpreter. Working with code that's not covered with any tests and you can't tell immediately if you broke it is never nice.
We could create something like test/libyul/yulInterpreterTests/isoltestTesting/ and put such tests there. We do that with other kinds of tests.
Collaborator
There was a problem hiding this comment.
I actually wanted to comment the same for #16267 earlier, but that was merged before I got to it :)
Member
Author
There was a problem hiding this comment.
Yep I agree, would be good.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Theoretically,
loadimmutableinserts a placeholder into the contract to be deployed which is then replaced bysetimmutableonce the runtime code is loaded into memory.The interpreter is mainly used for differential fuzzing so we should satisfy that correct code never fails and we can still - most of the time - detect bugs.
Therefore,
loadimmutableis decoupled fromsetimmutableand just returns a random value (hash of the identifier),setimmutableis treated as a no-op.